Skip to content

fix: adjust store icon 🐛#1047

Open
lethemanh wants to merge 1 commit intomasterfrom
adjust-store-icon
Open

fix: adjust store icon 🐛#1047
lethemanh wants to merge 1 commit intomasterfrom
adjust-store-icon

Conversation

@lethemanh
Copy link
Copy Markdown

@lethemanh lethemanh commented Jul 3, 2025

CleanShot 2025-07-03 at 11 15 56@2x

@lethemanh lethemanh self-assigned this Jul 3, 2025
@lethemanh lethemanh requested review from JF-Cozy and zatteo as code owners July 3, 2025 04:14
@bundlemon
Copy link
Copy Markdown

bundlemon bot commented Jul 3, 2025

BundleMon

Files updated (2)
Status Path Size Limits
static/js/cozy.(hash).js
464.1KB (+43B +0.01%) -
static/js/main.(hash).js
271.25KB (+12B 0%) -
Unchanged files (8)
Status Path Size Limits
static/js/(chunkId).(hash).js
600.03KB -
services/autoDefineLabels.js
308.83KB -
services/keepIndexFullNameAndDisplayNameUpToD
ate.js
308.68KB -
services/cleanRelatedContacts.js
308.05KB -
static/js/lib-react.(hash).js
44.12KB -
static/css/main.(hash).css
30.52KB -
static/css/cozy.(hash).css
29.32KB -
static/js/lib-router.(hash).js
21.87KB -

Total files change +55B 0%

Groups updated (1)
Status Path Size Limits
**/*.js
3.67MB (+110B 0%) -
Unchanged groups (2)
Status Path Size Limits
**/*.{png,svg,ico}
1.14MB -
**/*.css
117.96KB -

Final result: ✅

View report in BundleMon website ➡️


Current branch size history | Target branch size history

{({ onClick, href }) => (
<ActionMenuItem
left={<AppIcon app="store" className="u-h-1 u-w-1" />}
left={<Icon icon={StoreIcon} className="u-h-1 u-w-1" />}
Copy link
Copy Markdown
Member

@zatteo zatteo Jul 3, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@JF-Cozy does <AppIcon/> mean that it is loading icon from "store" app but in stable release?

Copy link
Copy Markdown
Contributor

@JF-Cozy JF-Cozy Jul 3, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://github.com/cozy/cozy-ui/blob/master/react/AppIcon/index.jsx#L53 then https://github.com/cozy/cozy-client/blob/master/packages/cozy-stack-client/src/getIconURL.js#L144 so from stack (by default) or registry.

So I think we have to dig in these directions and not changing anything here

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok right, so it should be automatically fixed when updating store to stable release. I suggest we keep this PR until monday if we need it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants